Skip to content

Conversation

@petertseng
Copy link
Member

@petertseng petertseng commented Mar 18, 2018

jssander and others added 6 commits March 18, 2018 20:39
All implementing tracks have this function, but it is never mentioned.

exercism/problem-specifications#1195
I did not consider this to provide any additional value.

I asked for this to be removed in
exercism/problem-specifications#1195 (comment)
I requested this be added in
exercism/problem-specifications#1195 (review)

There was an implicit test of this in the 5x5 board, but we'll start
small.

Note that this track deliberately does not change `territories` to be a
map mapping a player to all of their territories, instead keeping it an
array of all separate contiguous regions. This is because I (as the
track maintainer) believe that keeping it an array provides better
fidelity of the board and helps ensure that implementations are doing
the right thing.
exercism/problem-specifications#1195

Note that this track deliberately does not change `territories` to be a
map mapping a player to all of their territories, instead keeping it an
array of all separate contiguous regions. This is because I (as the
track maintainer) believe that keeping it an array provides better
fidelity of the board and helps ensure that implementations are doing
the right thing.
@petertseng petertseng merged commit f9a14ba into exercism:master Mar 18, 2018
@petertseng petertseng deleted the go-counting branch March 18, 2018 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants